From: Florian Müllner <fmuellner@gnome.org>
Date: Wed, 11 Aug 2021 22:33:33 +0000 (+0200)
Subject: appMenu: Remove separator between "new window" and actions
X-Git-Tag: archive/raspbian/50.5-1+rpi1~1^2~574^2^2~101
X-Git-Url: https://dgit.raspbian.org/%22http://www.example.com/cgi/%22/%22http:/www.example.com/cgi/%22?a=commitdiff_plain;h=bba799a70c9292139ac4fbd810cbd183044638b6;p=gnome-shell.git

appMenu: Remove separator between "new window" and actions

We hide our own "New Window" item if the app itself includes a
"new-window" action. That means that the separator between the
built-in item and desktop actions introduces a small inconsistency
depending on whether a "New Window" item is provided by the desktop
action or ourselves. There's no good reason for that from a user's
perspective, so remove the separator.

Part-of: <https://gitlab.gnome.org/GNOME/gnome-shell/-/merge_requests/1948>
---

diff --git a/js/ui/appMenu.js b/js/ui/appMenu.js
index 9d1bcd30b..a5011cc31 100644
--- a/js/ui/appMenu.js
+++ b/js/ui/appMenu.js
@@ -57,8 +57,6 @@ var AppMenu = class AppMenu extends PopupMenu.PopupMenu {
             Main.overview.hide();
         });
 
-        this.addMenuItem(new PopupMenu.PopupSeparatorMenuItem());
-
         this._actionSection = new PopupMenu.PopupMenuSection();
         this.addMenuItem(this._actionSection);